Skip to content

Use tcx.stable_crate_id() more often#154464

Closed
zetanumbers wants to merge 1 commit intorust-lang:mainfrom
zetanumbers:more_stable_crate_ids
Closed

Use tcx.stable_crate_id() more often#154464
zetanumbers wants to merge 1 commit intorust-lang:mainfrom
zetanumbers:more_stable_crate_ids

Conversation

@zetanumbers
Copy link
Copy Markdown
Contributor

A small refactor of code like tcx.def_path_hash(cnum.as_def_id()).stable_crate_id() into tcx.stable_crate_id(cnum) which effectively has the same output.

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Mar 27, 2026
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Mar 27, 2026

r? @TaKO8Ki

rustbot has assigned @TaKO8Ki.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: compiler
  • compiler expanded to 69 candidates
  • Random selection from 12 candidates

@zetanumbers
Copy link
Copy Markdown
Contributor Author

Closed in favor of #154371

@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 27, 2026
@zetanumbers zetanumbers deleted the more_stable_crate_ids branch March 27, 2026 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants